controller/internal/service/controller_service.go (1)

941-959: Ordering difference from router_service.go.

In this file, user-configured options (s.ServerOptions) are placed first via append(s.ServerOptions, interceptors...), whereas router_service.go places defaults first and appends user options. While this likely doesn't cause functional issues (gRPC options are typically additive rather than overriding), consider aligning the ordering across both services for consistency and easier maintenance.

♻️ Optional: Align ordering with router_service.go

-	opts := append(s.ServerOptions,
+	opts := []grpc.ServerOption{
 		grpc.ChainUnaryInterceptor(func(
 			gctx context.Context,
 			req any,
 			_ *grpc.UnaryServerInfo,
 			handler grpc.UnaryHandler,
 		) (resp any, err error) {
 			return handler(logContext(gctx), req)
 		}, recovery.UnaryServerInterceptor()),
 		grpc.ChainStreamInterceptor(func(
 			srv any,
 			ss grpc.ServerStream,
 			_ *grpc.StreamServerInfo,
 			handler grpc.StreamHandler,
 		) error {
 			return handler(srv, &wrappedStream{ServerStream: ss})
 		}, recovery.StreamServerInterceptor()),
-	)
+	}
+	opts = append(opts, s.ServerOptions...)
 	server := grpc.NewServer(opts...)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@controller/internal/service/controller_service.go` around lines 941 - 959,
The append ordering of server options is inconsistent with router_service.go:
currently opts := append(s.ServerOptions, grpc.ChainUnaryInterceptor(...),
grpc.ChainStreamInterceptor(...)) places user options first; change it to
construct opts by starting with the default interceptors (the
grpc.ChainUnaryInterceptor and grpc.ChainStreamInterceptor entries) and then
append s.ServerOptions so defaults come first, ensuring the same ordering used
when creating the server via grpc.NewServer; update the block that builds opts
(references: s.ServerOptions, grpc.ChainUnaryInterceptor,
grpc.ChainStreamInterceptor, grpc.NewServer) accordingly.